Add AI contribution guidelines - #13369
Conversation
✅ Docs preview has no changesThe preview was not built because there were no changes. Build ID: 9fdb68c9e8077bd96f444a2b ✅ AI Style Review — No Changes DetectedNo MDX files were changed in this pull request. Review Log: View detailed log
|
|
|
Important Review skippedAuto incremental reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Repository UI (base), Organization UI (inherited) Review profile: CHILL Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
📝 WalkthroughWalkthrough
ChangesAI contribution guidelines
Estimated code review effort: 1 (Trivial) | ~3 minutes Suggested reviewers: Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
commit: |
There was a problem hiding this comment.
Actionable comments posted: 2
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@CONTRIBUTING.md`:
- Line 214: Update the contributor guidance sentence to use the correctly
hyphenated term “follow-up” instead of “followup,” without changing the rest of
the sentence.
- Line 215: Update the contribution guidance around “Do not ignore the issue
templates” so opening an issue first is required only for substantial or unclear
changes, not every PR. Preserve the existing exceptions and direct-PR guidance
for documentation improvements and small bug fixes, while keeping the big-PR
workflow’s issue-first requirement.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Repository UI (base), Organization UI (inherited)
Review profile: CHILL
Plan: Pro Plus
Run ID: 220e329b-f57e-43a4-a750-495e0fa1e3b4
📒 Files selected for processing (1)
CONTRIBUTING.md
17ff642 to
afbfe11
Compare
| - [Building](#building) | ||
| - [Testing](#testing) | ||
| - [Wiring a checkout into an application](#wiring-a-checkout-into-an-application) | ||
| - [AI contribution guidelines](#ai-contribution-guidelines) |
There was a problem hiding this comment.
We should also add a link to this in the PR templates.
| - Always review AI-generated code yourself before submitting a PR. | ||
| - Only open issues and PRs that you understand yourself. The maintainers tend to ask follow-up questions about the issue or design choices in a PR and you need to be able to explain them. | ||
| - Do not ignore the issue templates. Open an issue first to discuss solutions with the maintainers before opening PRs to reach agreement on how best to fix. The maintainers often have additional context about the code that might be relevant to the problem. | ||
| - Avoid AI-generated content in comments, especially in response to questions and feedback posted by the maintainers. LLMs tend to take maintainer feedback verbatim and make changes without additional discussion. This adds a lot of churn and maintainers' time to constantly re-review changes that might not adequately address the feedback. |
There was a problem hiding this comment.
Please also avoid using AI to rephrase your comments unless absolutely necessary. We much rather talk to someone in non-perfect English than talking to them through a machine without knowing someone is there at the other end.
Probably there's a better way to say this, but the important part here is "I'd rather talk to someone in broken English than to Claude impersonating someone".
There was a problem hiding this comment.
Good call. I tacked on part of that to the end of this sentence: e376078
|
I think this is solid for now. We can always revisit and revise as needed. |
We are starting to get more and more contributions that use AI tools. This PR adds a section in our contribution guide on the use of AI tools so that we have something to reference.
Summary by CodeRabbit